From: Marco Trevisan (Treviño) <mail@3v1n0.net>
Date: Thu, 4 Dec 2025 04:18:04 +0000 (+0100)
Subject: environment: Do not try to animate a St.Adjustment with unmapped actor
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~55^2^2~17
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=2bd0e8b2c0f1ae128b63ef62b58b23e483b6c29b;p=gnome-shell.git

environment: Do not try to animate a St.Adjustment with unmapped actor

We were already doing this for actual actors, but the case was
skipped in the St.Adjustment case.

However, as it is an animatable we can just use its related actor as
reference, and keep the same logic despite their actual object type

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3991>
(cherry picked from commit 4d56b51100df05032ee78c5fa556b4e003de2761)
---

diff --git a/js/ui/environment.js b/js/ui/environment.js
index f3b461ca6..8cf764e95 100644
--- a/js/ui/environment.js
+++ b/js/ui/environment.js
@@ -197,9 +197,12 @@ function _easeAnimatableProperty(animatable, propName, target, params) {
     // whether the transition should finish where it started
     const isReversed = autoReverse && numIterations % 2 === 0;
 
+    // The object is a Clutter.Animatable.
+    const actor = animatable.get_actor();
+
     // Copy Clutter's behavior for implicit animations, see
     // should_skip_implicit_transition()
-    if (animatable instanceof Clutter.Actor && !animatable.mapped)
+    if (!actor?.mapped)
         duration = 0;
 
     const prepare = () => {